Skip to content

docs(ci): record live aggregate ruleset - #318

Merged
Eli Pinkerton (wallstop) merged 2 commits into
masterfrom
codex/issue-310-aggregate-ruleset-docs
Jul 30, 2026
Merged

docs(ci): record live aggregate ruleset#318
Eli Pinkerton (wallstop) merged 2 commits into
masterfrom
codex/issue-310-aggregate-ruleset-docs

Conversation

@wallstop

@wallstop Eli Pinkerton (wallstop) commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • record ruleset 17663217 as requiring exactly CI Success and Unity CI Success
  • retain the 14 individual static contexts only as migration history
  • record PR ci: centralize Unity enrollment lifecycle #316, the exact trusted audit result, and the remaining second-runner host repair in session progress

Validation

  • npx prettier --write docs/runbooks/required-checks.md progress/session-178-unity-validation-only.md
  • npx markdownlint-cli2 docs/runbooks/required-checks.md
  • npx cspell --no-progress --no-summary docs/runbooks/required-checks.md progress/session-178-unity-validation-only.md
  • npm run validate:docs:strict
  • npm run validate:all
  • pre-commit run --files docs/runbooks/required-checks.md progress/session-178-unity-validation-only.md progress/session-178-unity-validation-only.md.meta
  • git diff --check

Closes #310


Note

Low Risk
Documentation-only updates to runbook and session progress; no workflow, ruleset, or application code changes.

Overview
Updates required-checks runbook and session 178 progress to match production: ruleset 17663217 now requires only CI Success and Unity CI Success (live 2026-07-30), with PR #316 cited as verification.

The runbook drops interim “15 individual contexts” guidance and the “aggregate not yet live” note. The 14 static job names are kept under Retired Individual Static Contexts for migration history and CI Success dependency audits, not for branch protection. The aggregate switch section is marked DONE with preserved conditions/bypass actors; fragile-check wording now states branch protection requires only CI Success.

Session progress replaces open PR/host-repair steps with delivery evidence: merge commits, all nine Unity matrix cells green (including 6000.3.16f1 standalone IL2CPP), and trusted org audit run 30587754261 with zero DxMessaging findings on the merged default-branch commit.

Reviewed by Cursor Bugbot for commit 62dfa58. Bugbot is set up for automated code reviews on this repo. Configure here.

Copilot AI review requested due to automatic review settings July 30, 2026 22:46
@wallstop

Copy link
Copy Markdown
Collaborator Author

Cursor (@cursor) review
Copilot review

Please verify that the runbook matches live ruleset 17663217 exactly and that the progress record states only evidence established by PR #316 and trusted audit run 30587754261.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Comment thread progress/session-178-unity-validation-only.md Outdated
Copilot AI review requested due to automatic review settings July 30, 2026 22:53
@wallstop

Copy link
Copy Markdown
Collaborator Author

Cursor (@cursor) review
Copilot review

The only reported issue is fixed in 62dfa58. Please re-review the current head.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 62dfa58. Configure here.

@wallstop
Eli Pinkerton (wallstop) merged commit 97a614b into master Jul 30, 2026
37 of 39 checks passed
@wallstop
Eli Pinkerton (wallstop) deleted the codex/issue-310-aggregate-ruleset-docs branch July 30, 2026 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ci: required-checks ruleset requires individual static job names the aggregate already covers

2 participants